-
-
Notifications
You must be signed in to change notification settings - Fork 188
Conversation
I would more like to see a PR that cover cli and non cli usage ( #73) |
Note: input and output do not have their place in a generic config file |
#73 looks like it belongs to another commit. This PR simply allows users to specify a config flag. Also you haven't settled on the file name in that issue. cli usage is covered in the unit tests. I will remove input and output from the config file. |
Done. (it seems appveyor failed to install microtime, not sure how that happened.) |
Can you also include documentation and a line in the changelog for that (as |
Changelog updated. By documentation do you mean the readme file? It already contains the |
Good point ! |
Thank you for accepting the PR. :) |
Released in 1.1.0 |
👏 |
Feel free to change the flag name and the
input
&output
options in the config file.